Hi,

first of all, nice module!

I do have a little question. When I use the Dynamic tree menu, it does not display correctly (I think...).

I will explain with an example.

Let's say you have the following taxonomy:
Continent A
- Country 1
-- AAA
- Country 2
-- BBB
Continent B
- Country 3

When I for example drill down on 'Continent A' in my dynamic tree menu, I will see that 'country 1' and 'country 2' are under 'Continent A' but I will also see that 'Conintent B' is placed on the same level as the countries under 'continent A'. And when I drill down on 'Country 2', 'Continent B' is placed on the same level as 'AAA'.

Is this supposed to be like this?
I yes, I don't get what the additional advantage is...

Thank you in advance!

CommentFileSizeAuthor
#3 Snippet.zip164.66 KBKerel

Comments

Kerel’s picture

Kerel commented

The avoid misunderstandings: The 'AAA' I mentioned should be 'BBB'.

styro’s picture

styro commented

Could you please look through the HTML output (eg in Firebug) to confirm whether this is a HTML nesting problem or a CSS theming problem. If it is HTML related, could you post a snippet of the output here to highlight the actual problem.

The dynamic menu was probably the most complex part of the Drupal 5 module - there is a good chance there are new bugs lurking in there since the Drupal 6 rewrite. It hasn't had a lot of testing so far.

Kerel’s picture

Kerel commented
StatusFileSize
new Snippet.zip164.66 KB

Styro,

thanks for your answer.

First of all: these more technical things are new for me so I hope I am answering well. Anyway, I learn stuff so that's good.
I think it is a nesting problem. See the attached printscreen in the zip file. (If you want me to put it differently here, please tell me)

If hope this answer is sufficient....

Kerel’s picture

Kerel commented

Hi Styro,

I installed the latest update for taxonomy filter.
The following has changed:

- The hierarchy has disappeared in the dynamic tree menu. So for a hierarchy, all the terms are now put on the same level (flat instead of a tree menu).
For a large taxonomy it would be more convenient to stick to the 'tree'.

I don't know if this was ment to be? Just to let you know.

solotandem’s picture

solotandem commented
Assigned: Unassigned » solotandem

Kerel: Try tonight's dev release (although I have not changed the dynamic tree menu code). I have made changes to base, tf_count, tf_multi, and tf_cloud. Next on my agenda will be the other two templates: dynamic and context. Maybe you can help test it out. Do you have a demo site I could look at?

Kerel’s picture

Kerel commented

Hi solotandem,

I am more than happy to do this testing. I will download the dev release tonight (I hope).
Problem is I do not have a demo site online... Will take some time to do this but is there another way of showing results in the meanwhile?
I will already post (as soon as I downl the new dev release) an attachment with all my findings that I think are worth mentioning.

solotandem’s picture

solotandem commented

Kerel: Tomorrow's dev release (dated May 17) will include the context and dynamic menu templates. Download it and test the dynamic menu. Thanks for your help.

Kerel’s picture

Kerel commented

Yes I noticed the templates were not in the previous dev version yet. I will download it one of the following days, do a test and put my findings here.

Kerel’s picture

Kerel commented

At first sight it looks perfect. The problem I mentioned before has been solved so that is a good thing.
And other things I tried also work.

I have not yet tested everything that can be tested but I will do that asap...

Good work!

solotandem’s picture

solotandem commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.